Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add data-cta-name to documentation #15392

Merged
merged 2 commits into from
Oct 30, 2024
Merged

Add data-cta-name to documentation #15392

merged 2 commits into from
Oct 30, 2024

Conversation

stephaniehobson
Copy link
Contributor

@stephaniehobson stephaniehobson commented Oct 25, 2024

One-line summary

Add data-cta-name as a supported attribute for CTAs, implement on WNP 132

Issue / Bugzilla link

#15391

Testing

make livedocs

WNP 132 is on demo2 for testing.

@stephaniehobson stephaniehobson added Needs Review Awaiting code review Review: µ Code review time: 5 minutes or less Analytics 📈 Documentation 📝 Things related to the bedrock documentation and removed Analytics 📈 labels Oct 25, 2024
Copy link
Member

@alexgibson alexgibson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

@alexgibson alexgibson merged commit 087e994 into main Oct 30, 2024
6 checks passed
@alexgibson alexgibson deleted the ga-cta-name branch October 30, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Analytics 📈 Documentation 📝 Things related to the bedrock documentation Needs Review Awaiting code review Review: µ Code review time: 5 minutes or less
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants